Fix typo in GtkIconPaintable is-symbolic property

icon-theme: Add back and fix test for handling non-square icons

We test this by looking at the produced render nodes now that
we don't actualluy scale the icon. Also, it turns out that this
code was broken due to some typos, so we also fix those.

Expose gtk_icon_paintable_new_for_file

GtkIconPaintable: Add properties for file, icon-name and is-symbolic

icon paintable: Replace get_filename and get_resource_path with get_file()

This returns a GFile which can represent both the above.

Rename GTK_ICON_LOOKUP_LOAD_IN_THREAD to GTK_ICON_LOOKUP_PRELOAD

icontheme: Always return some icon name in get_icon_name() for lookup results

If you called gtk_icon_theme_lookup(), then always return some useful
icon name from gtk_icon_paintable_get_icon_name(), even if we picked
an unthemed icon.

Also rewrite the gtk_icon_paintable_get_icon_name() docs to make this
clearer.

GtkIconHelper: Trigger preload when css sizes is valid

When the css is validated we know the css size, so we can
create the paintable at that point, and we do so passing the
LOAD_IN_THREAD flag.

This causes us to load most icons early on, in parallel instead of
during the first snapshot.

icon-theme: Add GTK_ICON_LOOKUP_LOAD_IN_THREAD flag

This causes lookup to trigger a thread that loads the icon texture.

icontheme: Optimize memory use and lookup speed by internalizing icon names

Instead of having the IconTheme have a hashtable that owns
individual strings and then IconThemeDirSize have a similar
hash (but with the strings owned by the other hash), we
have a consecutive memory chunks where we store the icon names
and then the hashtable has pointers into this.

This means we can avoid a bunch of individual strdup()s in a
way that is less fragmented and wastes less space. Additionally,
since we do an initial lookup anyway we have the internalized
icon name during lookup which means we can use g_direct_hash/equal
instead of g_str_hash/equal making the lookup faster too.

Remove stray space

Use G_OBJECT_TYPE_NAME instead of g_type_name_from_instance

Add gtk_icon_paintable_get_resource_path()

Also, return NULL from gtk_icon_paintable_get_filename() if the path
is really a resource path.

Add gtk_icon_paintable_get_icon_name()

This allows you to see which icon was actually chosen.

Remove gtk_icon_paintable_download_texture() public API

This allows us do do what we want internally, and anyway it didn't
really do what you'd expect anymore now that we do resizing during
rendering.

Replace last users of gtk_icon_paintable_download_texture ()

These now render the paintable to a cairo surface and convert that
to a texture. This is sort of a hack, but its only used in two
special cases internally and in two hacky test apps.

textview: Use paintables instead of textures, and fix the support

This changes gtk_text_buffer_insert_texture() to
gtk_text_buffer_insert_paintable() which is strictly more useful
(as textures are paintables). It also fixes the code to actually
support drawing the paintables (as well as tracking changes
to the paintables.

gtk_css_style_snapshot_icon_paintable: Fix uninitialized memory access

We were reading the foreground color alpha even when we didn't read it.

GskPango: Allow specifying a custom shape snapshot handler

We will need this for GtkTextView to render the textures in it.

GtkBuilder: Directly load the fallback missing image

No need for all this complexity with the icon theme, especially when
its now a lot more work to get the texture back. We can just load
the pixbuf.

icontheme: Move IMAGE_MISSING_RESOURCE_PATH to (private) header

This way other code can use this resource too for fallbacks, avoing
having to go through the icon theme.

icontheme: Handle GEmblemIcon in lookup_by_gicon()

icon theme: Also never fail in gtk_icon_theme_lookup_by_gicon()

Similar to regular lookups, if the icon is some unknown type we return
a missing-image.

Remove unnecessary image-missing fallbacks

We now always return something from icon theme lookups, so no need to
do custom fallbacks.

icon-theme: Don't pre-scale pixbufs in gtk_icon_theme_lookup_by_gicon

We do scaling during rendering now anyway.

IconTheme: Never fail a lookup or icon load

If icon lookup fails or if loading it fails later, just always
fall back to the built in image-missing icon. Nobody is handling
missing icons in a sane way anyway.

If you *truly* need to handle missing icons, you need to manually
use gtk_icon_theme_has_icon().

While changing the loading code I also fixed an issue where it
was always passing "png" to pixbuf, now it also handles "xpm" if
that is the filename suffix.

Support aspect=TRUE in _gdk_pixbuf_new_from_stream_at_scale

The icon theme test failed without this, and we *should* handle
it if we're accepting the argument.

IconTheme: Simplify icon scaling

We had a pretty complex setup where we tried to avoid scaling up themes from dirs
that specified a size. However, not only was it very complex, but it didn't quite
work with window scales, because when using e.g. a size 32 directory for 16@2x
the dir size is wrong anyway. Additionally it turns out most code either picks
an existing icon size, or uses the FORCE_SIZE flags, so it doesn't seem
like a useful behaviour.

This change drops the FORCE_SIZE flags, and always scales
icons. Additionally it moves the scaling of the icon to rendering,
which seems more modern, and allows us to (later) share icons loaded
for different sizes that happened to use the same source file (at
different scales).

Note that this changes the behaviour of
gtk_icon_paintable_download_texture() is it now returns the unscaled
source icon. However, ignore thats, as I plan to remove this function
and replace it with a way to render a paintable to a cairo-surface
instead.

GtkIconPaintable: Always recolor when snapshotting.

It never makes sense to paint a texture that needs recoloring. If
you call the regular snapshot on a symbolic texture we just use the
default recoloring colors.

To support this we also change gtk_css_style_snapshot_icon_paintable()
to call gtk_icon_paintable_snapshot_with_colors() for IconPaintables
so that we get the correct colors, and we make it not emit the color
matrix.

Since we now rely on the icon to do the recoloring we also drop the
recolor argument in gtk_icon_paintable_snapshot_with_colors() as its
not needed anymore.

icontheme: Align function arguments

Merge branch 'kill-simple-search' into 'master'

Remove simple search engine

See merge request GNOME/gtk!1393

Merge branch 'better-test-isolation-2' into 'master'

Better test isolation

See merge request GNOME/gtk!1391

css tests: Ignore ".csd" vs ".solid-csd"

It it hard to control which of the csd style classes we get,
since it depends on details of the X server or compositor.

Explicitly ignore this difference by replacing .solid-csd
with .csd in the output.

testsuite: Make tests run without a11y bus

Not having an a11y bus around causes warnings from
at-spi, which make tests fail. Explicitly ignore those
warnings.

testsuite: Fix bloomfilter style tests

The results were expecting solid-csd, which is not what
we usually have.

Merge branch 'master.msvc' into 'master'

Fix building recent GTK master on Visual Studio

See merge request GNOME/gtk!1369

Merge branch 'public-emojichooser' into 'master'

emojichooser: Make public

See merge request GNOME/gtk!1382

Merge branch 'wip/otte/css' into 'master'

Some CSS cleanup

See merge request GNOME/gtk!1389

emojichooser: Add profiler marks for loading

This is a long-running operation, and worth showing
in traces.

icontheme: Fix various memory leaks

Tasks need to be unreffed too.

widget: Fix a memory leak with class actions

This code was doing the wrong thing even if the parent class
doesn't have actions.

Add a lib64 variant of gtk.supp

icontheme: Remove some dead code

frameclock: cosmetics

Redo the profiler marks for the frame clock signals
to look more uniform.

gdk: Silence some compiler warnings

These cause unused variable warnings in release builds.

treestore: Silence compiler warnings

This was causing compiler warnings in release builds.

notebook: Don't trigger criticals in dispose

When we dismantle our children in dispose, we
trigger a11y children-changed signals which end
up calling back into the notebook. Handle this
without critical warnings.

textview: Don't update action state in dispose

Only update the action state when we have a text buffer,
there is no need to do it otherwise, since we are going
to get a buffer before we get shown. This avoids triggering
the action state updates from finalize, which is a bad
time to be recreating the action muxer.

entry: Avoid criticals in dispose

The accessible gets properties of the entry, and
resetting the entry icons triggers accessible change
notification, so do that before we dismantle the entry
too far to respond to a g_object_get () call.

Don't export color picker functions

This is not public API, so it shouldn't be exported.

Read bookmarks async

Do not block the main thread while reading the bookmarks file.
This speeds up initial setup of a GtkPlacesSidebar.

placessidebar: Change some initial property values

Giving these properties the initial value that the file
chooser wants for them avoids some calls to update_places
in the setup code.

testperf: More flexible mark selection

Allow selecting marks not just by their name, but
also by the message detail.

testperf: Allow measuring start time

This can be used e.g. to measure how long it takes us
to get the first frame on screen.

stylecontext: Remove unused typedef

css: Make some local variables const

css: Remove unused function

style context: Remove parent relationship

We don't have foreign drawing anymore, so we don't need this.

stylecontext: Remove gtk_style_context_new()

It's private now, no more foreign drawing of any sort.

To be fair, it was entirely useless without widget paths already.

testsuite: Remove the stylecontext test

Stylecontexts are on their way out and I'm removing API that the
testsuite was relying on, so remove the tests.

Put the useful parts of the tests elsewhere.

stylecontext: Remove the "changed" signal

Instead, always go directly to the GtkWidget::css_changed() call.

widget: Replace style-updated signal with css_changed vfunc

1. Rename the thing
2. Turn it from a signal to a vfunc
3. Pass the GtkCssStyleChange to it

We don't export any public API about the GtkCssStyleChange yet, it's
just a boring opaque struct.

pathbar: Remove unneeded code

The icons here are GIcons, they don't depend on the theme.

cssnode: Remove redundant check

We assert that cssnode->parent is NULL, no need to check it again.

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1388

widget: Skip not-drawable widgets in snapshot_child

Otherwise we transform the snapshot just to not do anything.

maplistmodel: Add an example to the docs

treelistmodel: Small documentation tweaks

widgetfocus: Remove unnecessary check

Mapped widgets are always realized.

widget: Remove _gtk_widget_is_drawable()

Use _gtk_widget_get_mapped() instead.

Drawable checked visible && mapped and mapped widgets are always
visible, so that was an unnecessary check.

Merge branch 'wip/otte/icontheme' into 'master'

Do simple icontheme API cleanups

See merge request GNOME/gtk!1378

Rename GtkIcon to GtkIconPaintable

tooltip: Don't css-validate

That's a job for toplevels now.

See 17ca95a1617b009a76b27a8572671855435985e7 for the same comit on
popovers.

icontheme: Remove GENERIC_FALLBACKS

Instead, rely on people passing fallbacks explicitly.

Alternatively, GThemedIcon provides the functionality to create
fallbacks, which is what GtkImage and the testsuite now use.
That method is slightly better, too, so the expected test results
have been updated accordingly.

icontheme: Pass fallbacks as optional argument to lookup_icon()

This way, we can remove gtk_icon_theme_choose_icon() completely.

icontheme: Remove contexts

There is no way to query contexts or do anything useful with them.

So don't keep track of them and don't make them an argument in public
APIs with the docs saying "I don't know what to use here, maybe read
some spec somewhere".

icontheme: Clean up includes

icontheme: Remove gtk_icon_get_base_size() and gtk_icon_get_base_dir()

Those functions are unused and the documentation says "Returns some
random number that the icon theme creator chose" which does not seem at
all useful and an implementation detail.

So get rid of it.

Revert "Break out cairo color matrix recoloring to gdk_cairo_image_surface_recolor()"

We're not in the business of adding Cairo APIs. That's Cairo's job.

Also, we don't need this API anywhere like the original commit claimed,
so there's no need to make it available in any way.

This reverts commit afa6cc23692f83f0d38de24c9cf58a7a494c01d2.

icontheme: Remove crayon APIs

We expose no API to get at any colors for drawing symbolics, so we
shouldn't have APIs to draw with them.

Apart from that, those APIs look like a box of crayons, not like an
icontheme.

icontheme: Remove gtk_icon_theme_rescan_if_needed()

If the icon theme needs to be rescanned, it should just do it.
If it doesn't, there's no need for this function.

Ergo: Don't have the function.

icontheme: Remove NO_SVG and FORCE_SVG flags

If the icon theme loads SVGs or not is an implementation detail and
should not be exposed in public API.

icontheme: Remove GTK_ICON_LOOKUP_USE_BUILTIN

It's unused.

icontheme: Make text direction a lookup argument

Most users were just forgetting to set the proper flags.
And flags aren't the right way to set this anyway, it was just
acceptable as a workaround during GTK3 to not break API.

icontheme: Remove async APIs

Widgets would not use them properly. In fact, the only user was using
them wrong.

As icons are loaded async by default, this call isn't necessary.

icontheme: Remove gtk_icon_theme_get_default()

The API encouraged wrong usage - most of the users were indeed wrong.

Use the correct version instead:
  gtk_icon_theme_get_for_display (gtk_widget_get_display ())

a11y tests: Ignore "active" flag on toplevel

All the a11y tests were failing for me with a window state diff
like this:

-  state: active enabled resizable sensitive showing visible
+  state: enabled resizable sensitive showing visible

I guess the windows in the CI always gets the focus, but not when
I run it here. Generally focus seems asynchronous and hard to rely
on so I just made the test ignore the active state on toplevels.

tests: Initialize dbus before gtk

These days initilizing gtk may create a connection to the sesson bus,
so we have to initialize GTestDBus before initalizing gtk, or we'll
use the address of the "real" session bus (and remember that in the
global).

To further muck things up, g_test_dbus_up() resets important env
vars like DISPLAY and XDG_RUNTIME_DIR, which we have to re-set.

filechooser portal: Free portal proxy when bus connection closes

This was blocking the clean exit from the testdbus shutdown in
the defaultvalues test. The proxy was keeping the connection alive
which blocks g_test_dbus_down().

css: Fix the initial values optimization

Fix up the index computation. We have duplicate entries
in the type enum, so to go from one of the 'initial' types
to it corresponding type you subtract one, but to find
the size array entry for a type, you divide by 2.

Merge branch 'faster-theme-lookup' into 'master'

Faster icon theme lookup

See merge request GNOME/gtk!1381

icon-theme: Only have one copy of the icon flags enum

It turns out with the icon cache now using the virtual SYMBOLIC_PNG_SUFFIX
flag the two enums are now identical, so lets just use one of them, the
one GtkIconCache (so we move it to the header).

Don't use xsettings or xft defaults in testsuite

This adds a GDK_DEBUG=default-settings flag which disables reads
from xsettings and Xft resources, and enables this for the testsuite.

This is one less way to get different testresults depending on the
environment. In particular, it was failing the css tests for me
due to getting the wrong font size because i have a different dpi.

emojichooser: Make public

This is a useful widget to have, and it has minimal api.

Not having it public forces apps to recreate a lot of
complicated machinery for not good reason, if they need
an Emoji chooser in a different context.

GtkIconCache: Remove unused code

We're not really using the icon theme cache much anymore, as the
individual per-directory hashes are no longer used, so delete all the
unused code.

IconTheme: Make icon lookups faster

Traditionally the icon lookup for a theme has been:

lookup (icon_name, size):
  best_directory = NULL
  forearch theme
    foreach directory in theme
       if dir_size_matches (directory, size) && dir_has_icon (directory, icon-name)
         best_directory = chose_best_size_dir (best_directory, directory)

if best_directory
  return icon from best_directory

However, it turns out that there are a lot of subdirectories which have the same
size, as they differ only in the (essentially useless) "context" value. For example
the "16x16/apps" subdirectory is essentially the same as the "16x16/actions" one.

So, instead rathern than keeping all the directories as separate we store the
all the directories with the same size as a single entity (DirSize) and the
icon lookup in that DirSize looks up not only which suffix to use for that icon
but also which subdir it is in.

Additionally we keep a hashtable with all icon names that are
available in the entire theme (i.e. all DirSizes), which allows use
both to store each icon name only once, but also to do a quick
negative lookup and early exit in case we're looking up an icon that
doesn't exist. This is pretty common because we often look up sets of
icons like "image-png-symbolic", "image-png", "image", expecting some
to fail.

This brings down the time of the initial css validation from 20msec to 15msec for
me when running icon-factory.

icon-cache: Add new function to list all the icons in a directory

This lists the icons in a particular director, with their flags in a
hashtable. We also convert from "icon.symbolic" + SUFFIX_PNG to
"icon" + SUFFIX_SYMBOLIC_PNG.

meson/MSVC builds: Use -utf-8 where available

This avoids the build from erroring out on C4819 (Unicode handling issue in
Visual Studio compiler), notably when running on Chinese, Japanese and
Korean locales.

Also apply -D_USE_MATH_DEFINES, -FImsvc_recommended_pragmas.h and -utf-8 to
the C++ compiler options as well.

Remove last GtkWidgetPath reference

See 49b47c913389dc395a48c2b19ac409412623c524

demos/widget-factory/widget-factory.c: Include stdlib.h

Include it for exit().